Skip to content
This repository was archived by the owner on Mar 1, 2019. It is now read-only.

slack対応#8

Open
bonk wants to merge 9 commits into
c-bata:masterfrom
bonk:develop
Open

slack対応#8
bonk wants to merge 9 commits into
c-bata:masterfrom
bonk:develop

Conversation

@bonk

@bonk bonk commented May 4, 2017

Copy link
Copy Markdown

プレゼンで使ってみようと思いまして、
slackのchannelを拾って表示できるようにしてみました。

試しにpull requestを作ってみました。
初心者ですいません。失礼があればすいません。

slack対応以外に次のことをしました。

  • コードを少しフォーマットしました。
  • トレイ周りが自分の環境ですぐ落ちてしまっていたので変更しました。
  • 自分の環境で重たくなってしまったのでループ処理をsetTimeoutにしました。
// このあたりです。
let tray = null;

.envに設定を記載する必要が出ます。

SLACK_BOT_TOKEN="ほげ"
SLACK_BOT_CHANNEL="ほげ"
STREAM_TYPE="slack"
STREAM_SPEED="100"
STREAM_DX_BASE="-5"
STREAM_LOOP_TYPE="setTimeout"

kawahara and others added 6 commits May 3, 2017 14:09
* install stylelint
* install dotenv & sdk
* install crash reporter
* resume tray
@c-bata

c-bata commented May 4, 2017

Copy link
Copy Markdown
Owner

Slack対応面白いですね!少しレビュー遅くなるかもしれませんが、必ず試してみます!

* fix about dotenv
* fix about resource
@bonk

bonk commented May 4, 2017

Copy link
Copy Markdown
Author

ぎゃー、返事ありがとうございます。うれしい。
ちなみに、プルリクの後で追加でパッケージングできるように調整を入れました。m(_ _)m

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants